apps: Replace CONFIG_ARCH_HAVE_VFORK with CONFIG_ARCH_HAVE_FORK #1962
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
xiaoxiang781216
approved these changes
Aug 13, 2023
masayuki2009
force-pushed
the
apps_fork
branch
from
976ff11
to
987c2e4
Compare
masayuki2009
added a commit
to masayuki2009/incubator-nuttx
that referenced
this pull request
Nov 16, 2023
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
masayuki2009
force-pushed
the
apps_fork
branch
from
987c2e4
to
a720d66
Compare
Summary: - Resolves an issue where `vfork_test` was not executed following the renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK, as detailed in apache/nuttx#9755. Impact: - Please merge apache/nuttx#11200 before merging this PR. Testing: - Successfully tested with rv-virt:smp64, spresense:smp, and sim:smp. Signed-off-by: Masayuki Ishikawa <[email protected]>
masayuki2009
force-pushed
the
apps_fork
branch
from
a720d66
to
31f1c00
Compare
|
Please ignore the following style issues. |
xiaoxiang781216
pushed a commit
to apache/nuttx
that referenced
this pull request
Nov 16, 2023
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in #9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
xiaoxiang781216
approved these changes
Nov 16, 2023
casaroli
pushed a commit
to casaroli/nuttx
that referenced
this pull request
Jan 25, 2024
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
freakishness
pushed a commit
to freakishness/incubator-nuttx
that referenced
this pull request
Feb 18, 2024
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
halyssonJr
pushed a commit
to halyssonJr/nuttx
that referenced
this pull request
Apr 10, 2024
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
nealef
pushed a commit
to nealef/nuttx
that referenced
this pull request
May 29, 2024
Summary: - Temporarily disables ARCH_HAVE_FORK for arm64 to address a crash issue, as detailed in apache#9755. - This is a precautionary measure until a more permanent solution is implemented. Impact: - Temporarily limits certain functionalities on arm64, but necessary to ensure system stability. Testing: - Successfully tested on QEMU-8.1.2. - Note: please apply the changes from apache/nuttx-apps#1962. Signed-off-by: Masayuki Ishikawa <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
vfork_testwas not executed followingthe renaming from CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK,
as detailed in sched/task/fork: add fork implementation nuttx#9755.
Impact
merging this PR.
Testing